ModelActions.js ➔ deselectAll - Code Metrics - Inspection of "add selectRow and deselectRow actions" - bencripps/react-redux-grid - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 559b30...4c24b0 )

by Benjamin

created 2016-10-12 14:37 UTC

ModelActions.js ➔ deselectAll A

↳ Parent: src/actions/plugins/selection/ModelActions.js

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
c	0
b	0
f	0
nc	1
dl	0
loc	3
rs	10
nop	1

import {
    SET_SELECTION,
    SELECT_ALL,
    DESELECT_ALL,
    NO_EVENT,
    SELECT_ROW,
    DESELECT_ROW
} from '../../../constants/ActionTypes';

export const selectRow = ({ stateKey, rowId }) => ({
    type: SELECT_ROW,
    stateKey,
    rowId
});

export const deselectRow = ({ stateKey, rowId }) => ({
    type: DESELECT_ROW,
    stateKey,
    rowId
});

export const selectAll = ({ data, stateKey }) => {

    if (!data) {
        return {};
    }

    const keys = data.currentRecords.map((row) => row._key);

    const selection = keys.reduce((obj, k) => {
        obj[k] = true;
        return obj;
    }, {});

    return { type: SELECT_ALL, selection, stateKey };
};

export const deselectAll = ({ stateKey }) => ({
    type: DESELECT_ALL, stateKey
});

export const setSelection = ({
    id, defaults = {}, modes = {}, stateKey, index
}) => {

    const allowDeselect = defaults.allowDeselect;
    const clearSelections = defaults.mode === modes.checkboxSingle
        || defaults.mode === modes.single;

    if (!defaults.enabled) {
        /* eslint-disable no-console */
        console.warn('Selection model has been disabled');
        /* eslint-enable no-console */
        return { type: NO_EVENT };
    }

    if (defaults.mode === modes.single) {
        return {
            type: SET_SELECTION,
            id,
            clearSelections,
            allowDeselect,
            index,
            stateKey
        };
    }

    else if (defaults.mode === modes.multi) {
        return { type: SET_SELECTION, id, allowDeselect, index, stateKey };
    }

    else if (defaults.mode === modes.checkboxSingle
function isBig(a) {
    if (a > 5000) {
        return "yes";
    }
}

console.log(isBig(5001)); //returns yes
console.log(isBig(42)); //returns undefined
        || defaults.mode === modes.checkboxMulti) {
        return {
            type: SET_SELECTION,
            id,
            clearSelections,
            allowDeselect,
            index,
            stateKey
        };
    }
};


1			import {
2			SET_SELECTION,
3			SELECT_ALL,
4			DESELECT_ALL,
5			NO_EVENT,
6			SELECT_ROW,
7			DESELECT_ROW
8			} from '../../../constants/ActionTypes';
9
10			export const selectRow = ({ stateKey, rowId }) => ({
11			type: SELECT_ROW,
12			stateKey,
13			rowId
14			});
15
16			export const deselectRow = ({ stateKey, rowId }) => ({
17			type: DESELECT_ROW,
18			stateKey,
19			rowId
20			});
21
22			export const selectAll = ({ data, stateKey }) => {
23
24			if (!data) {
25			return {};
26			}
27
28			const keys = data.currentRecords.map((row) => row._key);
29
30			const selection = keys.reduce((obj, k) => {
31			obj[k] = true;
32			return obj;
33			}, {});
34
35			return { type: SELECT_ALL, selection, stateKey };
36			};
37
38			export const deselectAll = ({ stateKey }) => ({
39			type: DESELECT_ALL, stateKey
40			});
41
42			export const setSelection = ({
43			id, defaults = {}, modes = {}, stateKey, index
44			}) => {
45
46			const allowDeselect = defaults.allowDeselect;
47			const clearSelections = defaults.mode === modes.checkboxSingle
48			\|\| defaults.mode === modes.single;
49
50			if (!defaults.enabled) {
51			/* eslint-disable no-console */
52			console.warn('Selection model has been disabled');
53			/* eslint-enable no-console */
54			return { type: NO_EVENT };
55			}
56
57			if (defaults.mode === modes.single) {
58			return {
59			type: SET_SELECTION,
60			id,
61			clearSelections,
62			allowDeselect,
63			index,
64			stateKey
65			};
66			}
67
68			else if (defaults.mode === modes.multi) {
69			return { type: SET_SELECTION, id, allowDeselect, index, stateKey };
70			}
71
72			else if (defaults.mode === modes.checkboxSingle
			0 ignored issues – show Complexity Best Practice introduced 2016-09-08 18:04 UTC by Report Bug Copy Issue Report There is no return statement if `defaults.mode === modes....=== modes.checkboxMulti` is `false`. Are you sure this is correct? If so, consider adding `return;` explicitly. This check looks for functions where a `return` statement is found in some execution paths, but not in all. Consider this little piece of code function isBig(a) { if (a > 5000) { return "yes"; } } console.log(isBig(5001)); //returns yes console.log(isBig(42)); //returns undefined The function `isBig` will only return a specific value when its parameter is bigger than 5000. In any other case, it will implicitly return `undefined`. This behaviour may not be what you had intended. In any case, you can add a `return undefined` to the other execution path to make the return value explicit. Loading history...
73			\|\| defaults.mode === modes.checkboxMulti) {
74			return {
75			type: SET_SELECTION,
76			id,
77			clearSelections,
78			allowDeselect,
79			index,
80			stateKey
81			};
82			}
83			};
84

bencripps / react-redux-grid

GitHub Access Token became invalid

Push — master ( 559b30...4c24b0 )

ModelActions.js ➔ deselectAll A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like